Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fixing java lesson's test output #277

Merged
merged 11 commits into from
Nov 1, 2024
Merged

Conversation

unsigable
Copy link
Contributor

@unsigable unsigable commented Sep 16, 2024

  1. Add output example for 22nd java lesson's task (like it was in previous lessons)

  2. Fix output:

System.setOut(new PrintStream(new FileOutputStream(FileDescriptor.out)));
System.out.println(actual);

@unsigable unsigable changed the title Fixing 22nd java lesson's output Fixing java lesson's output Sep 22, 2024
@unsigable
Copy link
Contributor Author

unsigable commented Sep 22, 2024

Fixing tests with simple results = single string output

...
final var expected = "result";
...

@unsigable unsigable changed the title Fixing java lesson's output Fixing java lesson's test output Sep 23, 2024
@unsigable
Copy link
Contributor Author

Внес правки для устранения:

exercises-1 | Test.java:17: error: cannot find symbol
exercises-1 | System.setOut(new PrintStream(new FileOutputStream(FileDescriptor.out)));
exercises-1 | symbol: variable FileDescriptor
exercises-1 | location: class Test
exercises-1 | 2 errors

@unsigable
Copy link
Contributor Author

Fixing another 2 simple outputs (+checked the modified files for presence of FileDescriptor)

@fey fey requested a review from Malcom1986 September 30, 2024 13:16
@sgmdlt
Copy link
Contributor

sgmdlt commented Oct 1, 2024

@Malcom1986

ByteArrayOutputStream out = new ByteArrayOutputStream();
System.setOut(new PrintStream(out));

App.printMotto();
App.main(null);
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

А почему имя метода поменяли?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

По не опытности и не знанию.
(еще только учусь)
Исправлю!

@Malcom1986
Copy link
Contributor

@unsigable Привет! Есть один вопросик, написал выше

@unsigable
Copy link
Contributor Author

unsigable commented Oct 8, 2024

@Malcom1986 Возвращаю как было!
Спасибо за замечание!)

Если будут другие - учту!
Я стараюсь мониторить Issue каждый день.

P.S.
В прошлый раз почему-то вышла ошибка из-за верификации почты - до этого с таким не сталкивался!
Не понимаю почему периодически приходят запросы на повторную верификацию !?
(Не хотелось бы создавать Вам проблем из-за таких мелочей)

@Malcom1986
Copy link
Contributor

@unsigable Привет! Поправили? Не вижу нового коммита

@Malcom1986
Copy link
Contributor

@unsigable Привет! Нужна тук какая-нибудь помощь?

@unsigable
Copy link
Contributor Author

@unsigable Привет! Поправили? Не вижу нового коммита

@Malcom1986 Прошу прощение за задержку - обстоятельства...

Я внёс правку - вернул название метода printMotto.

 ByteArrayOutputStream out = new ByteArrayOutputStream();
 System.setOut(new PrintStream(out));

 App.printMotto();

 final var actual = out.toString().trim();

Что-то еще надо было исправить?

@unsigable
Copy link
Contributor Author

@unsigable Привет! Нужна тук какая-нибудь помощь?

у меня почему-то периодически начали возникать проблемы то с "подтверждением почты", то с O2Auth...
в настройках уже несколько раз всё менял.

@Malcom1986 Malcom1986 merged commit a477387 into hexlet-basics:main Nov 1, 2024
1 check failed
@unsigable unsigable deleted the test branch November 2, 2024 08:28
@unsigable unsigable restored the test branch November 2, 2024 08:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants